Skip to content

Conversation

ariostas
Copy link
Contributor

This PR changes the links to the training center so that they point directly there instead of getting there through a series of redirections. The Plausible Analytics for the training center says that most people are getting there with a direct link, but I suspect that the reason for this is that the redirections are obscuring the real sources. So we will see if after making this change we see that most people are really coming from the HSF website.

Copy link

netlify bot commented Apr 21, 2025

Deploy Preview for earnest-hotteok-b1e1bf ready!

Name Link
🔨 Latest commit 40ce0cc
🔍 Latest deploy log https://app.netlify.com/sites/earnest-hotteok-b1e1bf/deploys/6806960e713100000864b230
😎 Deploy Preview https://deploy-preview-1728--earnest-hotteok-b1e1bf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ariostas
Copy link
Contributor Author

I fixed a link that was broken, but the other two broken links reported by the Check Markdown links workflow are false positives. They probably just rate-limited GitHub's servers.

@ariostas
Copy link
Contributor Author

@michmx I can't request a reviewer, but could you take a look at this when you get a chance?

@michmx
Copy link
Contributor

michmx commented Apr 28, 2025

I fixed a link that was broken, but the other two broken links reported by the Check Markdown links workflow are false positives. They probably just rate-limited GitHub's servers.

For the record

ERROR: 2 dead links found!
[✖] https://mattermost.com/ → Status: 403
[✖] https://demo.codimd.org/ → Status: 403

Both links active with pages up and running.

Copy link
Contributor

@michmx michmx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fixes!

@michmx michmx merged commit c4acfd5 into HSF:main Apr 28, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants